Mobile: add workflow publishing UI (parity with VSCode)#99
Merged
Conversation
PublishForm gains the skill/workflow toggle + owned-skills checklist the VSCode webview builder already has: workflow mode swaps the SKILL.md box for a checklist of owned skills, synthesizes type: workflow + requiredSkills frontmatter on submit, and validates 1-16 selections (MAX_REQUIRED_SKILLS). Required dependency: surfaces/localhost's emitOwnedSkills didn't attach workflowMints, so the mobile picker had no way to exclude owned workflows (a workflow can't require another workflow, per the on-chain gate). Wired workflowMintsAmong (now exported from core) through the ownedSkills event, same ground-truth on-chain read the VSCode side uses instead of relying on card.type (unreliable when a mint is missing from the indexer catalog). Also: Publish now opens pre-set to whichever tab you were browsing (skill/workflow), matching the VSCode entry point. Fixes #96 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
25 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
packages/core/src/chat/ui/webview.ts, commit279b8e6) to the mobilePublishForm(surfaces/webview): a skill/workflow toggle swaps the SKILL.md box for an owned-skills checklist, which synthesizestype: workflow+requiredSkillsfrontmatter on submit through the existingpublishSkillpath.surfaces/localhost'semitOwnedSkillsnow attachesworkflowMints(viaworkflowMintsAmong, newly exported from core) so the mobile picker excludes owned workflows — same on-chain ground-truth read VSCode uses, not the catalog-dependentcard.type.MAX_REQUIRED_SKILLS).kind/requiredSkillsfields to thepublishSkillmessage for forward-compat with Workflow publishing: contract + backend + webview/VSCode/mobile + CLI + MCP tool (#88) #95's contract path, once merged.Acceptance criteria (from #96)
Test plan
tsc --noEmitclean onpackages/core,surfaces/localhost,surfaces/webview(no new errors vs. baseline)vite buildsucceeds forsurfaces/webviewFixes #96
🤖 Generated with Claude Code